Skip to content

Stash the active backend in quark_queue_stats{}#84

Merged
haesbaert merged 1 commit intomainfrom
pimp-stats
Oct 21, 2024
Merged

Stash the active backend in quark_queue_stats{}#84
haesbaert merged 1 commit intomainfrom
pimp-stats

Conversation

@haesbaert
Copy link
Collaborator

The whole idea of "the user should not even know which backend is being used", worked a bit too much.

This was prompted by a bug in beats where seccomp was blocking a syscall needed for EBPF, and so it fell back to kprobe all the time and the user had no proper way to notice this.

Related to elastic/beats#41297

Purposedly stashed this into stats to make it hard to retrieve from quark.c, if we start doing if (qq->backend == QQ_EBPF) we lost the war.

The whole idea of "the user should not even know which backend is being used",
worked a bit too much.

This was prompted by a bug in beats where seccomp was blocking a syscall needed
for EBPF, and so it fell back to kprobe all the time and the user had no proper
way to notice this.

Related to elastic/beats#41297

Purposedly stashed this into stats to make it hard to retrieve from quark.c,
if we start doing `if (qq->backend == QQ_EBPF)` we lost the war.
@haesbaert haesbaert requested a review from a team as a code owner October 18, 2024 08:21
Copy link
Contributor

@nicholasberlin nicholasberlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@haesbaert haesbaert merged commit fea80ba into main Oct 21, 2024
@haesbaert haesbaert deleted the pimp-stats branch November 12, 2024 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants